From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Thu, 4 Dec 2025 04:18:04 +0000 (+0100)
Subject: environment: Do not try to animate a St.Adjustment with unmapped actor
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~44^2~52
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=4d56b51100df05032ee78c5fa556b4e003de2761;p=gnome-shell.git

environment: Do not try to animate a St.Adjustment with unmapped actor

We were already doing this for actual actors, but the case was
skipped in the St.Adjustment case.

However, as it is an animatable we can just use its related actor as
reference, and keep the same logic despite their actual object type

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3991>
---

diff --git a/js/ui/environment.js b/js/ui/environment.js
index 369f9de49..094b95018 100644
--- a/js/ui/environment.js
+++ b/js/ui/environment.js
@@ -197,9 +197,12 @@ function _easeAnimatableProperty(animatable, propName, target, params) {
     // whether the transition should finish where it started
     const isReversed = autoReverse && numIterations % 2 === 0;
 
+    // The object is a Clutter.Animatable.
+    const actor = animatable.get_actor();
+
     // Copy Clutter's behavior for implicit animations, see
     // should_skip_implicit_transition()
-    if (animatable instanceof Clutter.Actor && !animatable.mapped)
+    if (!actor?.mapped)
         duration = 0;
 
     const prepare = () => {